Hello! 👋🏻 I ran into this bug while using multiple useInView hooks with merged refs, so I opened an issue and put together this fix.

Let me know if anything needs to be changed!

Loving the library by the way : )

Description

Fixes #746

This PR fixes a bug where callbacks are skipped when multiple useInView({ triggerOnce: true }) hooks are merged on the same element.

Problem

When a callback executes with triggerOnce: true, it calls unobserve() which removes the callback from the array using splice(). This modifies the array during forEach iteration, causing subsequent callbacks to be skipped.

Solution

Copy the callbacks array before iterating using the spread operator:

[...elements.get(entry.target) ?? []].forEach((callback) => {
  callback(inView, entry);
});

This prevents the iteration from being affected by array modifications.

Changes

• src/observe.ts: Use spread operator to copy callbacks array before iteration
• src/tests/useInView.test.tsx: Add test case to verify all callbacks are triggered with triggerOnce and merged refs

Testing

Added a new test that reproduces the bug:

• Creates 3 useInView hooks with triggerOnce: true
• Merges all refs onto a single element
• Verifies all 3 hooks receive inView: true

Before fix: Test fails - only hooks 1 and 3 trigger, hook 2 remains false

After fix: Test passes - all 3 hooks trigger correctly : )